Skip to content

fix(cli): codex streaming merge, redaction fallbacks, transcript settle - #65

Merged
Patel230 merged 2 commits into
mainfrom
feat/cli-quality-improvements
Aug 12, 2026
Merged

fix(cli): codex streaming merge, redaction fallbacks, transcript settle#65
Patel230 merged 2 commits into
mainfrom
feat/cli-quality-improvements

Conversation

@Patel230

Copy link
Copy Markdown
Contributor

Remediation of trace findings:

  • CodexAgent gains PrepareTranscript: settle-on-stability wait for rollout transcripts (best-effort, never aborts the turn); 5 tests
  • compactCodex merges streaming assistant fragments with the same message ID per the documented contract (accumulative re-emitted + chunk-only variants, tool calls retained); 3 subtests
  • redaction failures now fall back to plain regex layers instead of storing EMPTY snapshot bytes (finalize + condensation paths); drop-on-failure tests updated to assert the fallback content
  • 4.2 (hook exit-0) and 4.4 (subagent baseline) verified already satisfied at HEAD — audited, no change needed

Verified: GOWORK=off go build/vet/test ./... green (full suite).

- CodexAgent gains PrepareTranscript (settle-on-stability wait for rollout
  transcripts; best-effort, never aborts the turn)
- compactCodex merges streaming assistant fragments with the same message
  ID per the documented contract (accumulative + chunk-only variants)
- redaction failures fall back to plain regex layers instead of storing
  EMPTY snapshot bytes (finalize + condensation paths)
- regression tests for all three
@Patel230
Patel230 merged commit a31e98c into main Aug 12, 2026
13 checks passed
@Patel230
Patel230 deleted the feat/cli-quality-improvements branch August 12, 2026 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant